From: Florian Müllner Date: Mon, 19 Aug 2019 19:22:20 +0000 (+0200) Subject: kbdA11yDialog: Avoid unnecessary ternary operators X-Git-Tag: archive/raspbian/50.5-1+rpi1~1^2~811^2^2~121 X-Git-Url: https://dgit.raspbian.org/%22http://www.example.com/cgi/%22/%22http:/www.example.com/cgi/%22?a=commitdiff_plain;h=fc964f975a8e40e079698f6bae626e6580236f7d;p=gnome-shell.git kbdA11yDialog: Avoid unnecessary ternary operators A condition is already boolean, there's no point in explicitly turning it into true/false. https://gitlab.gnome.org/GNOME/gnome-shell/merge_requests/710 --- diff --git a/js/ui/kbdA11yDialog.js b/js/ui/kbdA11yDialog.js index 26efbce0c..ba3576599 100644 --- a/js/ui/kbdA11yDialog.js +++ b/js/ui/kbdA11yDialog.js @@ -27,7 +27,7 @@ class KbdA11yDialog extends GObject.Object { if (whatChanged & Clutter.KeyboardA11yFlags.SLOW_KEYS_ENABLED) { key = KEY_SLOW_KEYS_ENABLED; - enabled = (newFlags & Clutter.KeyboardA11yFlags.SLOW_KEYS_ENABLED) ? true : false; + enabled = (newFlags & Clutter.KeyboardA11yFlags.SLOW_KEYS_ENABLED) > 0; title = enabled ? _("Slow Keys Turned On") : _("Slow Keys Turned Off"); @@ -36,7 +36,7 @@ class KbdA11yDialog extends GObject.Object { } else if (whatChanged & Clutter.KeyboardA11yFlags.STICKY_KEYS_ENABLED) { key = KEY_STICKY_KEYS_ENABLED; - enabled = (newFlags & Clutter.KeyboardA11yFlags.STICKY_KEYS_ENABLED) ? true : false; + enabled = (newFlags & Clutter.KeyboardA11yFlags.STICKY_KEYS_ENABLED) > 0; title = enabled ? _("Sticky Keys Turned On") : _("Sticky Keys Turned Off");